Skip to content

Allow raw delegated NyxID service calls - #3687

Merged
louis4li merged 3 commits into
feat/2026-09-18_nyxid-recommended-skill-readfrom
feat/2026-09-29_nyxid-raw-delegated-service
Sep 29, 2026
Merged

louis4li merged 3 commits into
feat/2026-09-18_nyxid-recommended-skill-readfrom
feat/2026-09-29_nyxid-raw-delegated-service

Conversation

@louis4li

Copy link
Copy Markdown
Contributor

Issue

Closes #3686

Scope

  • Add raw delegated mode to nyxid_invoke_operation when operation_id is omitted and method/relative_path are supplied.
  • Keep typed mode on live contracts when operation_id is present, and preserve existing document-guided mode compatibility.
  • Enforce service-relative paths, single visible service binding, credential selection, and sensitive header rejection before proxy execution.
  • Cover raw delegated reads for registration Agent Key and sender-bound credentials, plus unsafe path/header rejection.

Verification

  • dotnet test test/Aevatar.GAgents.ChannelRuntime.Tests/Aevatar.GAgents.ChannelRuntime.Tests.csproj --nologo --filter "ChannelNyxIdConnectedServiceInventoryToolSourceTests" passed: 48/48
  • bash tools/ci/test_stability_guards.sh passed
  • Local NyxID endpoint check not run; this change is covered at the channel tool/invoker boundary with fake NyxID proxy tests.

Notes

  • Raw delegated mode still routes through admitted proxy execution; it does not expose NyxID tokens or accept absolute URLs.
  • The base branch is feat/2026-09-18_nyxid-recommended-skill-read per implementation request.

Generated with Claude Code.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

louis4li and others added 2 commits September 29, 2026 20:00
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@louis4li
louis4li changed the base branch from feat/2026-09-18_nyxid-recommended-skill-read to feature/integrate September 29, 2026 12:48
@louis4li
louis4li changed the base branch from feature/integrate to feat/2026-09-18_nyxid-recommended-skill-read September 29, 2026 12:48
@louis4li
louis4li merged commit 27d9288 into feat/2026-09-18_nyxid-recommended-skill-read Sep 29, 2026
13 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant